Skip to content

Config option to disable previews#4426

Merged
elzody merged 2 commits intomainfrom
feat/1335/disable-preview
Mar 12, 2025
Merged

Config option to disable previews#4426
elzody merged 2 commits intomainfrom
feat/1335/disable-preview

Conversation

@juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented Jan 29, 2025

Implement #1335 and document our config flags

@juliusknorr juliusknorr requested a review from elzody as a code owner January 29, 2025 14:24
@juliusknorr juliusknorr added enhancement New feature or request 3. to review Ready to be reviewed labels Jan 29, 2025
@juliusknorr juliusknorr changed the title feat/1335/disable preview Config option to disable previews Jan 29, 2025
@juliusknorr juliusknorr force-pushed the feat/1335/disable-preview branch from 57af5b5 to a581f1c Compare January 31, 2025 20:04
@juliusknorr juliusknorr force-pushed the feat/1335/disable-preview branch from aea22a3 to aeeef49 Compare March 12, 2025 09:28
Signed-off-by: Julius Knorr <jus@bitgrid.net>
Signed-off-by: Julius Knorr <jus@bitgrid.net>
@juliusknorr juliusknorr force-pushed the feat/1335/disable-preview branch from effd8c1 to a4c920a Compare March 12, 2025 09:46
@elzody elzody merged commit 9941f7a into main Mar 12, 2025
82 of 86 checks passed
@elzody elzody deleted the feat/1335/disable-preview branch March 12, 2025 22:05
@KsmoinO
Copy link

KsmoinO commented Jul 14, 2025

Did this pull request planned to be included in a future release ?
This changes are still absent from richdocuments 8.7.2

@elzody
Copy link
Collaborator

elzody commented Jul 14, 2025

@KsmoinO It was never back-ported, so it was never included in any release. We do all development against the main branch and back-port fixes that should be included in current releases. Odds are, if this was never back-ported, it was intended to be included in the next major release (Nextcloud 32).

I am not clear on the process yet but I will see if this can be back-ported and included in the next maintenance releases of the app, since it doesn't seem like a major feature although it is considered an "enhancement."

@KsmoinO
Copy link

KsmoinO commented Jul 14, 2025

Thanks for your reply, it's more clear for me :)
Don't spend too much time on this as I have a workaround and I can wait for the next major release (even if I'd preferred don't have to patch after each updates :) )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Ready to be reviewed enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants